ComboBox: Really set list-mode popup transient-for
authorDaniel Boles <dboles@src.gnome.org>
Sun, 27 Aug 2017 23:12:02 +0000 (00:12 +0100)
committerDaniel Boles <dboles@src.gnome.org>
Mon, 28 Aug 2017 19:00:17 +0000 (20:00 +0100)
commitc789bf7718342b37464bc0d3beef6cb9449f0edc
treed0e20a346b4946b8dbddfc37299ce959ae1f5ac2
parent70534de382c827f690f1673a0b6710eff5dcfdf5
ComboBox: Really set list-mode popup transient-for

set_transient_for(toplevel) was only called in list_setup(). It was easy
to make a test showing a NULL :transient-for instead of the correct one.

So, move the call from list_setup() to popup_for_device(). Also do that
for window_group_add_window(), which means not calling it redundantly.

(I tried using a ComboBox:parent-set handler, but the Inspector’s CB
didn’t like that: it calls popup_for_device() twice and closes on button
release. Anyway, using popup() is much more concise than a new handler.)
gtk/gtkcombobox.c